-
Notifications
You must be signed in to change notification settings - Fork 1.4k
updated syncpack to v14 #7587
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updated syncpack to v14 #7587
Conversation
|
in all the version after |
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
Thanks for trying v14, just a note to be careful as it's still in alpha - any problems let me know and I can try to help. |
How to use config file? |
commit: |
Create a {
"$schema": "https://unpkg.com/[email protected]/schema.json"
// your config here
} EDIT: If you need them the docs have since been updated for v14 https://jamiemason.github.io/syncpack |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you confirm that the syntax works, or adjust?
"dependencies": [ | ||
"eslint" | ||
], | ||
"pinVersion": "^8.57.0 || ^9.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this syntax doesn't work, for vite we had to do >=5 <8
instead of 5||6||7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok,I didn't have the bandwidth to test those, closing this and creating a new PR with minimal changes
Closing this PR in favor of #7844 |
What is it?
Description
updated syncpack to v14, due to #7581
Checklist
pnpm change